From a5f8f5d9681bf38c9c080cb4662bbac8eb743f84 Mon Sep 17 00:00:00 2001
From: robertl <robertl@f51c46e8-681c-474f-0cfe-069cfd0219fb>
Date: Wed, 1 Jun 2011 00:54:54 +0000
Subject: [PATCH] Move document Lookat from bottom.  Thiw was blessed by Google
 long ago, but it's technically incorrect, so we fixed it.

git-svn-id: http://gpsbabel.googlecode.com/svn/trunk@4064 f51c46e8-681c-474f-0cfe-069cfd0219fb
---
 gpsbabel/kml.c                                | 27 ++++++++++++-------
 gpsbabel/reference/bounds-test.kml            | 10 +++----
 gpsbabel/reference/earth-expertgps-track.kml  | 18 ++++++-------
 gpsbabel/reference/earth-expertgps.kml        | 18 ++++++-------
 gpsbabel/reference/earth-gc.kml               | 18 ++++++-------
 .../reference/track/bounds-test-track.kml     | 10 +++----
 .../track/gpx_garmin_extensions-kml_track.kml | 18 ++++++-------
 .../reference/track/gtrnctr_power-kml.kml     | 18 ++++++-------
 .../track/segmented_tracks-track.kml          | 18 ++++++-------
 gpsbabel/reference/track/segmented_tracks.kml | 18 ++++++-------
 10 files changed, 91 insertions(+), 82 deletions(-)

diff --git a/gpsbabel/kml.c b/gpsbabel/kml.c
index 62b0667db..83634f114 100644
--- a/gpsbabel/kml.c
+++ b/gpsbabel/kml.c
@@ -772,11 +772,13 @@ static void kml_recompute_time_bounds(const waypoint *waypointp) {
   }
 }
 
-static void kml_output_point(const waypoint *waypointp, kml_point_type pt_type) {
-  const char *style;
-
+static void kml_add_to_bounds(const waypoint *waypointp) {
   waypt_add_to_bounds(&kml_bounds, waypointp);
   kml_recompute_time_bounds(waypointp);
+}
+
+static void kml_output_point(const waypoint *waypointp, kml_point_type pt_type) {
+  const char *style;
 
   switch (pt_type) {
     case kmlpt_track: style = "#track"; break;
@@ -1087,8 +1089,6 @@ static void kml_waypt_pr(const waypoint *waypointp)
 		kml_write_xml(-1, "</LookAt>\n");
 	}
 #endif
-	waypt_add_to_bounds(&kml_bounds, waypointp);
-	kml_recompute_time_bounds(waypointp);
 
 	if (waypointp->gc_data->diff && waypointp->gc_data->terr) {
 		kml_geocache_pr(waypointp);
@@ -1287,9 +1287,6 @@ static void kml_mt_hdr(const route_head *header)
     char time_string[64];
     waypoint *tpt = (waypoint *)elem;
 
-    // Add it to our bounding box so our default LookAt/flyto does a good
-    // thing.
-    waypt_add_to_bounds(&kml_bounds, tpt);
     if (tpt->creation_time) {
       xml_fill_in_time(time_string, tpt->creation_time, tpt->microseconds,
                        XML_LONG_TIME);
@@ -1390,6 +1387,17 @@ static void kml_route_tlr(const route_head *header)
 void kml_write_AbstractView(void) {
   double bb_size;
 
+  // Make a pass through all the points to find the bounds.
+  if (waypt_count()) {
+    waypt_disp_all(kml_add_to_bounds);
+  }
+  if (track_waypt_count())  {
+    track_disp_all(NULL, NULL, kml_add_to_bounds);
+  }
+  if (route_waypt_count()) {
+    route_disp_all(NULL, NULL, kml_add_to_bounds);
+  }
+
   kml_write_xml(1, "<LookAt>\n");
 
   if (kml_time_min || kml_time_max) {
@@ -1488,6 +1496,8 @@ void kml_write(void)
 		kml_write_xml(0, "<Snippet>Created %s</Snippet>\n", import_time);
 	}
 
+    kml_write_AbstractView();
+
 	// Style settings for bitmaps
 	if (route_waypt_count()) {
 		kml_write_bitmap_style(kmlpt_route, ICON_RTE, NULL);
@@ -1593,7 +1603,6 @@ void kml_write(void)
 		}
         }
 
-        kml_write_AbstractView();
 	kml_write_xml(-1, "</Document>\n");
 	kml_write_xml(-1, "</kml>\n");
 }
diff --git a/gpsbabel/reference/bounds-test.kml b/gpsbabel/reference/bounds-test.kml
index 047143904..be7a15ad8 100644
--- a/gpsbabel/reference/bounds-test.kml
+++ b/gpsbabel/reference/bounds-test.kml
@@ -3,6 +3,11 @@
 	xmlns:gx="http://www.google.com/kml/ext/2.2">
   <Document>
     <name>GPS device</name>
+    <LookAt>
+      <longitude>-117.144015</longitude>
+      <latitude>36.438270</latitude>
+      <range>87257.203848</range>
+    </LookAt>
 <!-- Normal route style -->
     <Style id="route_n">
       <IconStyle>
@@ -2164,10 +2169,5 @@
         </Placemark>
       </Folder>
     </Folder>
-    <LookAt>
-      <longitude>-117.144015</longitude>
-      <latitude>36.438270</latitude>
-      <range>87257.203848</range>
-    </LookAt>
   </Document>
 </kml>
diff --git a/gpsbabel/reference/earth-expertgps-track.kml b/gpsbabel/reference/earth-expertgps-track.kml
index 94a223e99..878b58cbb 100644
--- a/gpsbabel/reference/earth-expertgps-track.kml
+++ b/gpsbabel/reference/earth-expertgps-track.kml
@@ -3,6 +3,15 @@
 	xmlns:gx="http://www.google.com/kml/ext/2.2">
   <Document>
     <name>GPS device</name>
+    <LookAt>
+      <gx:TimeSpan>
+        <begin>2001-06-02T00:18:13Z</begin>
+        <end>2002-05-25T19:05:57Z</end>
+      </gx:TimeSpan>
+      <longitude>-81.356770</longitude>
+      <latitude>36.257086</latitude>
+      <range>2979727.763965</range>
+    </LookAt>
 <!-- Normal route style -->
     <Style id="route_n">
       <IconStyle>
@@ -3645,14 +3654,5 @@
         </Placemark>
       </Folder>
     </Folder>
-    <LookAt>
-      <gx:TimeSpan>
-        <begin>2001-06-02T00:18:13Z</begin>
-        <end>2002-05-25T19:05:57Z</end>
-      </gx:TimeSpan>
-      <longitude>-81.356770</longitude>
-      <latitude>36.257086</latitude>
-      <range>2979727.763965</range>
-    </LookAt>
   </Document>
 </kml>
diff --git a/gpsbabel/reference/earth-expertgps.kml b/gpsbabel/reference/earth-expertgps.kml
index be2a0ba7d..f389abdde 100644
--- a/gpsbabel/reference/earth-expertgps.kml
+++ b/gpsbabel/reference/earth-expertgps.kml
@@ -3,6 +3,15 @@
 	xmlns:gx="http://www.google.com/kml/ext/2.2">
   <Document>
     <name>GPS device</name>
+    <LookAt>
+      <gx:TimeSpan>
+        <begin>2001-06-02T00:18:13Z</begin>
+        <end>2002-05-25T19:05:57Z</end>
+      </gx:TimeSpan>
+      <longitude>-81.356770</longitude>
+      <latitude>36.257086</latitude>
+      <range>2979727.763965</range>
+    </LookAt>
 <!-- Normal route style -->
     <Style id="route_n">
       <IconStyle>
@@ -3477,14 +3486,5 @@
         </Placemark>
       </Folder>
     </Folder>
-    <LookAt>
-      <gx:TimeSpan>
-        <begin>2001-06-02T00:18:13Z</begin>
-        <end>2002-05-25T19:05:57Z</end>
-      </gx:TimeSpan>
-      <longitude>-81.356770</longitude>
-      <latitude>36.257086</latitude>
-      <range>2979727.763965</range>
-    </LookAt>
   </Document>
 </kml>
diff --git a/gpsbabel/reference/earth-gc.kml b/gpsbabel/reference/earth-gc.kml
index 667ef32f0..52b824d7c 100644
--- a/gpsbabel/reference/earth-gc.kml
+++ b/gpsbabel/reference/earth-gc.kml
@@ -3,6 +3,15 @@
 	xmlns:gx="http://www.google.com/kml/ext/2.2">
   <Document>
     <name>GPS device</name>
+    <LookAt>
+      <gx:TimeSpan>
+        <begin>2002-08-15T07:00:00Z</begin>
+        <end>2003-06-29T07:00:00Z</end>
+      </gx:TimeSpan>
+      <longitude>-79.930833</longitude>
+      <latitude>41.027500</latitude>
+      <range>2109328.437865</range>
+    </LookAt>
 <!-- Normal waypoint style -->
     <Style id="waypoint_n">
       <IconStyle>
@@ -155,14 +164,5 @@ Now that it's intuitively obvious to even the most casual observer where the cac
         </Point>
       </Placemark>
     </Folder>
-    <LookAt>
-      <gx:TimeSpan>
-        <begin>2002-08-15T07:00:00Z</begin>
-        <end>2003-06-29T07:00:00Z</end>
-      </gx:TimeSpan>
-      <longitude>-79.930833</longitude>
-      <latitude>41.027500</latitude>
-      <range>2109328.437865</range>
-    </LookAt>
   </Document>
 </kml>
diff --git a/gpsbabel/reference/track/bounds-test-track.kml b/gpsbabel/reference/track/bounds-test-track.kml
index b4d4b078b..3bb3be92b 100644
--- a/gpsbabel/reference/track/bounds-test-track.kml
+++ b/gpsbabel/reference/track/bounds-test-track.kml
@@ -3,6 +3,11 @@
 	xmlns:gx="http://www.google.com/kml/ext/2.2">
   <Document>
     <name>GPS device</name>
+    <LookAt>
+      <longitude>-117.144015</longitude>
+      <latitude>36.438270</latitude>
+      <range>87257.203848</range>
+    </LookAt>
 <!-- Normal track style -->
     <Style id="track_n">
       <IconStyle>
@@ -2281,10 +2286,5 @@
         </Placemark>
       </Folder>
     </Folder>
-    <LookAt>
-      <longitude>-117.144015</longitude>
-      <latitude>36.438270</latitude>
-      <range>87257.203848</range>
-    </LookAt>
   </Document>
 </kml>
diff --git a/gpsbabel/reference/track/gpx_garmin_extensions-kml_track.kml b/gpsbabel/reference/track/gpx_garmin_extensions-kml_track.kml
index e68071dc3..ee7c1d0d9 100644
--- a/gpsbabel/reference/track/gpx_garmin_extensions-kml_track.kml
+++ b/gpsbabel/reference/track/gpx_garmin_extensions-kml_track.kml
@@ -3,6 +3,15 @@
 	xmlns:gx="http://www.google.com/kml/ext/2.2">
   <Document>
     <name>GPS device</name>
+    <LookAt>
+      <gx:TimeSpan>
+        <begin>2008-08-20T07:04:48Z</begin>
+        <end>2008-08-20T07:04:55Z</end>
+      </gx:TimeSpan>
+      <longitude>-0.035329</longitude>
+      <latitude>51.506248</latitude>
+      <range>1300.000000</range>
+    </LookAt>
 <!-- Normal track style -->
     <Style id="track_n">
       <IconStyle>
@@ -420,14 +429,5 @@
         </Placemark>
       </Folder>
     </Folder>
-    <LookAt>
-      <gx:TimeSpan>
-        <begin>2008-08-20T07:04:48Z</begin>
-        <end>2008-08-20T07:04:55Z</end>
-      </gx:TimeSpan>
-      <longitude>-0.035329</longitude>
-      <latitude>51.506248</latitude>
-      <range>1300.000000</range>
-    </LookAt>
   </Document>
 </kml>
diff --git a/gpsbabel/reference/track/gtrnctr_power-kml.kml b/gpsbabel/reference/track/gtrnctr_power-kml.kml
index 358f78cf8..00bd762ae 100644
--- a/gpsbabel/reference/track/gtrnctr_power-kml.kml
+++ b/gpsbabel/reference/track/gtrnctr_power-kml.kml
@@ -3,6 +3,15 @@
 	xmlns:gx="http://www.google.com/kml/ext/2.2">
   <Document>
     <name>GPS device</name>
+    <LookAt>
+      <gx:TimeSpan>
+        <begin>2010-05-28T01:16:36Z</begin>
+        <end>2010-05-28T02:41:44Z</end>
+      </gx:TimeSpan>
+      <longitude>-122.139608</longitude>
+      <latitude>37.382794</latitude>
+      <range>19819.321245</range>
+    </LookAt>
 <!-- Normal track style -->
     <Style id="track_n">
       <IconStyle>
@@ -25688,14 +25697,5 @@
         </gx:Track>
       </Placemark>
     </Folder>
-    <LookAt>
-      <gx:TimeSpan>
-        <begin>2010-05-28T01:16:36Z</begin>
-        <end>2010-05-28T02:41:44Z</end>
-      </gx:TimeSpan>
-      <longitude>-122.139608</longitude>
-      <latitude>37.382794</latitude>
-      <range>19819.321245</range>
-    </LookAt>
   </Document>
 </kml>
diff --git a/gpsbabel/reference/track/segmented_tracks-track.kml b/gpsbabel/reference/track/segmented_tracks-track.kml
index b10c24d35..2fd6c296d 100644
--- a/gpsbabel/reference/track/segmented_tracks-track.kml
+++ b/gpsbabel/reference/track/segmented_tracks-track.kml
@@ -3,6 +3,15 @@
 	xmlns:gx="http://www.google.com/kml/ext/2.2">
   <Document>
     <name>GPS device</name>
+    <LookAt>
+      <gx:TimeSpan>
+        <begin>2007-07-27T05:24:05Z</begin>
+        <end>2007-07-27T05:35:00Z</end>
+      </gx:TimeSpan>
+      <longitude>-86.841461</longitude>
+      <latitude>35.831217</latitude>
+      <range>1944.793167</range>
+    </LookAt>
 <!-- Normal track style -->
     <Style id="track_n">
       <IconStyle>
@@ -1764,14 +1773,5 @@
         </Placemark>
       </Folder>
     </Folder>
-    <LookAt>
-      <gx:TimeSpan>
-        <begin>2007-07-27T05:24:05Z</begin>
-        <end>2007-07-27T05:35:00Z</end>
-      </gx:TimeSpan>
-      <longitude>-86.841461</longitude>
-      <latitude>35.831217</latitude>
-      <range>1944.793167</range>
-    </LookAt>
   </Document>
 </kml>
diff --git a/gpsbabel/reference/track/segmented_tracks.kml b/gpsbabel/reference/track/segmented_tracks.kml
index 94bc9d2bc..0afc7791d 100644
--- a/gpsbabel/reference/track/segmented_tracks.kml
+++ b/gpsbabel/reference/track/segmented_tracks.kml
@@ -3,6 +3,15 @@
 	xmlns:gx="http://www.google.com/kml/ext/2.2">
   <Document>
     <name>GPS device</name>
+    <LookAt>
+      <gx:TimeSpan>
+        <begin>2007-07-27T05:24:05Z</begin>
+        <end>2007-07-27T05:35:00Z</end>
+      </gx:TimeSpan>
+      <longitude>-86.841461</longitude>
+      <latitude>35.831217</latitude>
+      <range>1944.793167</range>
+    </LookAt>
 <!-- Normal track style -->
     <Style id="track_n">
       <IconStyle>
@@ -1149,14 +1158,5 @@
         </Placemark>
       </Folder>
     </Folder>
-    <LookAt>
-      <gx:TimeSpan>
-        <begin>2007-07-27T05:24:05Z</begin>
-        <end>2007-07-27T05:35:00Z</end>
-      </gx:TimeSpan>
-      <longitude>-86.841461</longitude>
-      <latitude>35.831217</latitude>
-      <range>1944.793167</range>
-    </LookAt>
   </Document>
 </kml>
-- 
2.30.2